SCIX-818 feat(AbstractSources): add access-level badges to full text sources#794
Merged
thostetler merged 1 commit intoadsabs:masterfrom Feb 10, 2026
Merged
SCIX-818 feat(AbstractSources): add access-level badges to full text sources#794thostetler merged 1 commit intoadsabs:masterfrom
thostetler merged 1 commit intoadsabs:masterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #794 +/- ##
========================================
+ Coverage 61.3% 61.4% +0.1%
========================================
Files 307 308 +1
Lines 35535 35609 +74
Branches 1553 1564 +11
========================================
+ Hits 21762 21836 +74
+ Misses 13737 13736 -1
- Partials 36 37 +1
🚀 New features to boost your workflow:
|
…sources Show "Available" (green) or "Login required" (yellow) pill badges next to each source group name in accordion view, and per-item tags in menu view. Institution links are excluded from access labeling. - Extract access label logic into accessLabel.ts helper - Add group-level Badge in accordion, per-item Tag in menu - Normalize AcademicCapIcon size to match PDF/HTML icons (24x24) - Style institution icon as subtle dark gray - Suppress redundant "INSTITUTION" type label from institution links - Add unit tests for access label helpers - Add integration tests for badge rendering in both views
Member
Author
|
@shinyichen this was in response to a user mentioning that one the desktop view of the site we don't have a clear indication of OPEN_ACCESS links. which is true, in ADS we have text in the popover, but in scix we're only indicating this via color. Anyway, I thought this was a little better design, but would like some feedback! |
shinyichen
approved these changes
Feb 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Full text sources previously relied on icon color alone (green vs gray)
to indicate access level, making it difficult to scan at a glance.